-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat (HIS): add HIS metrics checker #68
base: main
Are you sure you want to change the base?
Conversation
I think an useful information in that table would be if the metric can be computed for each file or has to take the whole project into account. |
That's a good idea! We could add a column that states the granularity of each metric. |
Additional information regarding the HIS metrics as defined by the consortium (v1.3.1 - last open specification version). |
be420a9
to
c3679fc
Compare
@josecm @miguelafsilva5 Changed the base script for the HIS metrics to give more flexibility to the script:
To try this locally use as e.g.: |
c3679fc
to
cb4f36b
Compare
Signed-off-by: Afonso Santos <[email protected]>
Signed-off-by: Afonso Santos <[email protected]>
Signed-off-by: Afonso Santos <[email protected]>
Signed-off-by: Afonso Santos <[email protected]>
Signed-off-by: Afonso Santos <[email protected]>
e298879
to
27fec24
Compare
27fec24
to
c1452fe
Compare
Signed-off-by: Daniel Oliveira <[email protected]>
c1452fe
to
4a2edbb
Compare
eede022
to
f0fba61
Compare
HIS Checker
This PR introduces a new feature on the ci, the HIS metrics checker.
The following table lists the HIS metrics to be implemented, indicating the threshold for each metric (in the
Description
column).The
PR
column contains the link to the PR where the metric check was implemented.Edited by @danielRep
The MISRA-related metrics are addressed by the misra check (and not this checker):
NOMV: Violations of the HIS subset of MISRA C rules
NOMVP: Violations of the HIS subset of MISRA C rules by rule
The following metrics are currently not addressed since they are related to project evolution and cannot be evaluated in a single commit/snapshot:
SCHG: Number of statements changed in project
SDEL: Number of statements deleted in project
SI: Stability index of project
SNEW: Number of statements added in project
This PR will be used to discuss the overall implementation of the HIS checker and not technical code corrections of each metric (that should be done on the metric PR).
TODO